Skip to content

fix(ui5-time-picker): apply AM/PM selection on Enter key#13097

Merged
hinzzx merged 2 commits intomainfrom
tp-am-pm-keyboard-selection-fix
Mar 10, 2026
Merged

fix(ui5-time-picker): apply AM/PM selection on Enter key#13097
hinzzx merged 2 commits intomainfrom
tp-am-pm-keyboard-selection-fix

Conversation

@hinzzx
Copy link
Contributor

@hinzzx hinzzx commented Feb 16, 2026

Overview

When selecting AM/PM from the SegmentedButton using the Enter key, the picker closed but the value was not updated.

What We Did

  • Added period change handling before closing the picker when Enter is pressed on the AM/PM SegmentedButton

What This Fixes

  • ✅ Pressing Enter on AM/PM SegmentedButton now correctly updates the time value
  • ✅ No functional changes to other keyboard interactions

## Overview

When selecting AM/PM from the SegmentedButton using the Enter key, the picker closed but the value was not updated.

## What We Did

- Added period change handling before closing the picker when Enter is pressed on the AM/PM SegmentedButton

## What This Fixes

- ✅ Pressing Enter on AM/PM SegmentedButton now correctly updates the time value
- ✅ No functional changes to other keyboard interactions
@ui5-webcomponents-bot
Copy link
Collaborator

ui5-webcomponents-bot commented Feb 16, 2026

🧹 Preview deployment cleaned up: https://pr-13097--ui5-webcomponents.netlify.app

@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview February 16, 2026 14:18 Inactive
@didip1000 didip1000 self-requested a review February 19, 2026 08:04
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview February 27, 2026 08:06 Inactive
@hinzzx hinzzx merged commit a4710b3 into main Mar 10, 2026
21 of 22 checks passed
@hinzzx hinzzx deleted the tp-am-pm-keyboard-selection-fix branch March 10, 2026 08:08
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview March 10, 2026 08:08 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants